-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Core] Change token refresh condition and retry upserts #1215
[Core] Change token refresh condition and retry upserts #1215
Conversation
1. Update the condition upon which the JWT token is refreshed so it will refresh on expiration instead of only after. 2. Set upsert entenies as retryable. 3. Remove timeout in favor of global timeout. 4. Update `handle_request` to use method for indentifying retryable requests.
CHANGELOG.md
Outdated
- Remove timeout in favor of global timeout. (Change token refresh condition and retry upserts) | ||
- Update `handle_request` to use method for indentifying retryable requests. (Change token refresh condition and retry upserts) | ||
- Set upsert entenies as retryable. (Change token refresh condition and retry upserts) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove (Change token refresh condition and retry upserts)
CHANGELOG.md
Outdated
|
||
### Bug Fixes | ||
|
||
- Remove timeout in favor of global timeout. (Change token refresh condition and retry upserts) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove specific timeout for search request in favor of global timeout
Bump ocean version.
acffc38
to
04b4bbd
Compare
|
||
- Remove specific timeout for search request in favor of global timeout. | ||
- Update `handle_request` to use method for indentifying retryable requests. | ||
- Set upsert entenies as retryable. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Set upsert entenies as retryable. | |
- Set upsert entities as retryable. |
Description
What -
handle_request
to use method for identifying retryable requests.Why -
User receives 401 errors because JWT is expired so entities are not inserted.
Type of change
Please leave one option from the following and delete the rest:
All tests should be run against the port production environment(using a testing org).
Core testing checklist
Integration testing checklist
examples
folder in the integration directory.Preflight checklist
Screenshots
Include screenshots from your environment showing how the resources of the integration will look.
API Documentation
Provide links to the API documentation used for this integration.